Skip to content

ref(replays/issues): update replay inline onboarding panel to use prompt dismiss #69525

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 24, 2024

Conversation

michellewzhang
Copy link
Member

@michellewzhang michellewzhang commented Apr 23, 2024

image

@michellewzhang michellewzhang requested a review from a team April 23, 2024 19:12
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 23, 2024
Copy link
Member

@malwilley malwilley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding snoozing to this hook! Totally missed that when I added it

@michellewzhang michellewzhang force-pushed the mz/prompt-dismiss branch 2 times, most recently from d6a7c18 to d1ef0bb Compare April 23, 2024 20:44
Copy link

codecov bot commented Apr 23, 2024

Codecov Report

Attention: Patch coverage is 40.00000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 79.64%. Comparing base (129c9fc) to head (d1ef0bb).
Report is 21 commits behind head on master.

❗ Current head d1ef0bb differs from pull request most recent head 71095e6. Consider uploading reports for the commit 71095e6 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #69525      +/-   ##
==========================================
- Coverage   79.64%   79.64%   -0.01%     
==========================================
  Files        6479     6481       +2     
  Lines      287655   287694      +39     
  Branches    49580    49590      +10     
==========================================
+ Hits       229110   229135      +25     
- Misses      58108    58188      +80     
+ Partials      437      371      -66     
Files Coverage Δ
...events/eventReplay/replayInlineOnboardingPanel.tsx 78.78% <60.00%> (-1.22%) ⬇️
static/app/actionCreators/prompts.tsx 60.86% <20.00%> (-4.99%) ⬇️

... and 77 files with indirect coverage changes

@michellewzhang michellewzhang merged commit 907d497 into master Apr 24, 2024
42 checks passed
@michellewzhang michellewzhang deleted the mz/prompt-dismiss branch April 24, 2024 21:13
MichaelSun48 pushed a commit that referenced this pull request Apr 25, 2024
…mpt dismiss (#69525)

- followup from #69513
- closes #69207
- update the issue details replay inline onboarding CTA to use prompt
dismiss instead of `useDismissAlert`
- behavior should still be the same: snooze is 7 days, dismiss is
forever
- also made some updates to `usePrompt` hook to allow for snoozing


![image](https://github.com/getsentry/sentry/assets/56095982/93541900-5df6-4da9-ab14-0af54cda7575)
@github-actions github-actions bot locked and limited conversation to collaborators May 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Issue Details] Save Changes of the Session Replay CTA settings (snooze, and dismiss)
3 participants